Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: remove blue border in firefox #746

Merged
merged 2 commits into from
Oct 29, 2024
Merged

fix: remove blue border in firefox #746

merged 2 commits into from
Oct 29, 2024

Conversation

aritro2002
Copy link
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

When clicking on the button, or active iframe in Firefox, a blue border appears. This is caused by default browser behavior.
To resolve this, I have made every tags outline transparent when they are in focus and active state. The CSS change is applied specifically for Firefox to avoid affecting other browsers.

Reference:

  1. https://support.mozilla.org/en-US/questions/1401415
  2. https://tetralogical.com/blog/2023/01/13/foundations-visible-focus-styles/

Before:
Screenshot 2024-10-23 at 2 57 12 PM
After:
Screenshot 2024-10-23 at 2 57 21 PM

How did you test it?

Tested via rendering the SDK in firefox ,safari and chrome

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

Copy link

Review changes with SemanticDiff.

@PritishBudhiraja PritishBudhiraja added the Ready for Review PR with label Ready for Review should only be reviewed. label Oct 23, 2024
Copy link
Collaborator

@PritishBudhiraja PritishBudhiraja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good :)

@PritishBudhiraja PritishBudhiraja merged commit 40ef7a5 into main Oct 29, 2024
4 checks passed
@PritishBudhiraja PritishBudhiraja deleted the fix/firefox branch October 29, 2024 07:00
@github-actions github-actions bot added Closed Label will be automatically added when the PR will get merged to main and removed Ready for Review PR with label Ready for Review should only be reviewed. labels Oct 29, 2024
seekshiva pushed a commit that referenced this pull request Oct 29, 2024
## [0.96.3](v0.96.2...v0.96.3) (2024-10-29)

### Bug Fixes

* remove blue border in firefox ([#746](#746)) ([40ef7a5](40ef7a5))
aritro2002 added a commit that referenced this pull request Oct 30, 2024
@aritro2002 aritro2002 mentioned this pull request Oct 30, 2024
10 tasks
PritishBudhiraja pushed a commit that referenced this pull request Oct 30, 2024
PritishBudhiraja pushed a commit that referenced this pull request Oct 30, 2024
## [0.98.2](v0.98.1...v0.98.2) (2024-10-30)

### Reverts

* remove changes of pr [#746](#746) ([#758](#758)) ([cdae75e](cdae75e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Paypal issue When the redirected page closes, a blue border appears on iframe
2 participants